Cappl 3 ph #15125

patrickhuie19 · 2024-11-05T17:04:39Z

Requires

Supports

Comment to build plugins Updated Use goreleaser for compute WASM artifacts

* Prep @chainlink/contracts v1.3.0 release * Update contracts release date for v1.3.0

* Revert "fix: pinning goreleaser version to fix builds (#15108)" This reverts commit 8bbff53. * fix: arm64 dist directory

* adds OZ AccessControl support to the registry module * [Bot] Update changeset file with jira issues * fix snap * update version --------- Co-authored-by: app-token-issuer-infra-releng[bot] <120227048+app-token-issuer-infra-releng[bot]@users.noreply.github.com>

github-actions · 2024-11-05T17:06:19Z

Static analysis results are available

Hey @cedric-cordenier, you can view Slither reports in the job summary here or download them as artifact here.
Please check them before merging and make sure you have addressed all issues.

github-actions · 2024-11-06T11:49:26Z

AER Report: CI Core

aer_workflow , commit , Detect Changes , Clean Go Tidy & Generate , Scheduled Run Frequency , Flakeguard Root Project / Get Tests To Run , lint , Core Tests (go_core_tests) , Core Tests (go_core_tests_integration) , Core Tests (go_core_ccip_deployment_tests) , Core Tests (go_core_race_tests) , Core Tests (go_core_fuzz) , Flakeguard Deployment Project , Flakeguard Root Project / Run Tests (github.com/smartcontractkit/chainlink/v2/core/services/job, ubuntu-latest) , Flakeguard Root Project / Run Tests (github.com/smartcontractkit/chainlink/v2/core/services/standardcapabilities, ubuntu-la... , Flakeguard Root Project / Run Tests (github.com/smartcontractkit/chainlink/v2/core/services/workflows, ubuntu-latest) , Flakey Test Detection , SonarQube Scan , Flakeguard Root Project / Report

1. Redeclared variables in test file: [Run tests with flakeguard]

Source of Error:

core/services/standardcapabilities/standard_capabilities_test.go:10:2: require redeclared in this block
core/services/standardcapabilities/standard_capabilities_test.go:8:2: other declaration of require
core/services/standardcapabilities/standard_capabilities_test.go:10:2: "github.com/test-go/testify/require" imported and not used
core/services/standardcapabilities/standard_capabilities_test.go:20:2: logger redeclared in this block
core/services/standardcapabilities/standard_capabilities_test.go:17:2: other declaration of logger
core/services/standardcapabilities/standard_capabilities_test.go:20:2: "github.com/smartcontractkit/chainlink/v2/core/logger" imported and not used
core/services/standardcapabilities/standard_capabilities_test.go:21:2: job redeclared in this block
core/services/standardcapabilities/standard_capabilities_test.go:18:2: other declaration of job
core/services/standardcapabilities/standard_capabilities_test.go:21:2: "github.com/smartcontractkit/chainlink/v2/core/services/job" imported and not used

Why: The test file standard_capabilities_test.go has multiple redeclarations of variables (require, logger, job) and unused imports, causing compilation errors.

Suggested fix: Remove the redundant declarations and ensure each variable is declared only once. Also, remove any unused imports.

2. fmt.Printf format error: [Run tests]

Source of Error:

core/services/gateway/handlers/capabilities/handler.go:79:2: fmt.Printf format %d reads arg #2, but call has 1 arg

Why: The fmt.Printf function call expects two arguments for the format string %d, but only one argument is provided.

Suggested fix: Ensure that the number of arguments matches the format specifiers in the fmt.Printf call.

3. Replacement transaction underpriced: [Run tests]

Source of Error:

integration_test.go:96: 
	Error Trace:	/home/runner/work/chainlink/chainlink/core/services/transmission/integration_test.go:96
	 				/home/runner/work/chainlink/chainlink/core/services/transmission/integration_test.go:223
	Error: 	Received unexpected error:
	 	replacement transaction underpriced
	Test: 	Test4337WithLinkTokenPaymaster

Why: The error indicates that a replacement transaction was attempted with a gas price lower than the original transaction, which is not allowed.

Suggested fix: Increase the gas price for the replacement transaction to be higher than the original transaction's gas price.

4. CSA key does not exist: [Run tests with flakeguard]

Source of Error:

chainlink/application.go:711	[Feeds Service] Failed to start CSA key does not exist

Why: The Feeds Service failed to start because the required CSA key is missing.

Suggested fix: Ensure that the CSA key is generated and available before starting the Feeds Service.

5. SQL transaction aborted: [Run tests with flakeguard]

Source of Error:

EVM.0.Txm.Reaper	txmgr/reaper.go:79	unable to reap old txes: TxmReaper#reapEthTxes batch delete of finalized evm.txes failed: ReapTxes failed to delete old finalized evm.txes: ERROR: current transaction is aborted, commands ignored until end of transaction block (SQLSTATE 25P02)

Why: The SQL transaction was aborted, causing subsequent commands to be ignored.

Suggested fix: Investigate the cause of the transaction abort and ensure that the transaction is properly handled or retried.

6. Post request EOF error: [Run tests with flakeguard]

Source of Error:

cltest.go:778: 
	Error Trace:	/home/runner/work/chainlink/chainlink/core/internal/cltest/cltest.go:799
	 				/home/runner/work/chainlink/chainlink/core/internal/cltest/cltest.go:778
	 				/home/runner/work/chainlink/chainlink/core/internal/cltest/cltest.go:952
	 				/home/runner/work/chainlink/chainlink/core/services/job/runner_integration_test.go:1004
	Error: 	Received unexpected error:
	 	Post "http://127.0.0.1:38759/v2/external_initiators": EOF
	Test: 	TestRunner_Error_Callback_AsyncJob

Why: The POST request to the external initiator endpoint failed with an EOF error, indicating a premature end of the response.

Suggested fix: Ensure the external initiator service is running and accessible. Check for network issues or service crashes that might cause the EOF error.

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

github-actions · 2024-11-07T14:58:07Z

Flaky Test Detector for `./go.mod` project has failed ❌

Ran new or updated tests between develop and 2744b82 (CAPPL-3-ph).

View Flaky Detector Details | Compare Changes

Failed Tests

Ran 51 tests in total for all affected test packages. Below are the tests identified as flaky, with a pass ratio lower than the 100% threshold:

TestPackage                                                             TestName                                     PassRatio  RunCount   Skipped
---------                                                               ---------                                    ---------  ---------  ---------
github.com/smartcontractkit/chainlink/v2/core/services/workflows/store  Test_StoreDB_GetUnfinishedSteps              0%         1          false
github.com/smartcontractkit/chainlink/v2/core/services/workflows        TestEngine_GracefulEarlyTermination          0%         1          false
github.com/smartcontractkit/chainlink/v2/core/services/workflows        TestEngine_GetsNodeInfoDuringInitialization  0%         1          false
github.com/smartcontractkit/chainlink/v2/core/services/workflows        TestEngine_TimesOutOldExecutions             0%         1          false
github.com/smartcontractkit/chainlink/v2/core/services/workflows        TestEngine_MultiStepDependencies             0%         1          false
github.com/smartcontractkit/chainlink/v2/core/services/workflows        TestEngine_ResumesPendingExecutions          0%         1          false
github.com/smartcontractkit/chainlink/v2/core/services/workflows        TestEngine_WrapsTargets                      0%         1          false
github.com/smartcontractkit/chainlink/v2/core/services/workflows        TestRetryableSuccessAfterRetries             0%         1          false
github.com/smartcontractkit/chainlink/v2/core/services/workflows        TestRetryableErrorAfterMultipleRetries       0%         1          false

This reverts commit 86ab654.

This reverts commit 64aeef9.

This reverts commit ae63cc6.

…15383)" This reverts commit fa78941.

This reverts commit 1168674.

This reverts commit 86ab654.

This reverts commit 64aeef9.

This reverts commit 935612e.

justinkaseman and others added 6 commits November 5, 2024 14:52

Trigger build with CRON & KVSTORE standard capability binaries

dad2833

Comment to build plugins Updated Use goreleaser for compute WASM artifacts

@chainlink.contracts release v1.3.0 (#14812)

95dbcca

* Prep @chainlink/contracts v1.3.0 release * Update contracts release date for v1.3.0

fix: arm64 builds with goreleaser 2.4.x (#15110)

8800b27

* Revert "fix: pinning goreleaser version to fix builds (#15108)" This reverts commit 8bbff53. * fix: arm64 dist directory

Remove shallow checkout for changeset detection (#15113)

c3b5d37

removing dot seperator in path (#15120)

9ac0d6b

cedric-cordenier and others added 4 commits November 5, 2024 19:14

Add workflows

645858a

Add workflows

13474d1

Add missing workflows extra_files

cf987bf

Try optional copy tmp/workflows

5f7884a

justinkaseman added the build-publish Build and Publish image to SDLC label Nov 5, 2024

trigger build

96b0cfc

Create dummy registry

0d7ff26

cedric-cordenier force-pushed the CAPPL-3-ph branch from a316f57 to 0d7ff26 Compare November 6, 2024 11:58

Update workflow

c8d7b2f

cedric-cordenier force-pushed the CAPPL-3-ph branch from 6c60e67 to c8d7b2f Compare November 6, 2024 15:52

justinkaseman and others added 3 commits November 6, 2024 12:31

give hardcoded workflow a name + id

17680cf

Merge branch 'develop' into CAPPL-3-ph

fe4e07c

Bump common

444ffa0

cedric-cordenier force-pushed the CAPPL-3-ph branch from c03bf75 to 444ffa0 Compare November 7, 2024 11:52

cedric-cordenier added 2 commits November 7, 2024 12:11

Correct logger

399ed46

Incorporate KeyId into workflow

235d85b

cedric-cordenier force-pushed the CAPPL-3-ph branch from afd9d00 to 6eba838 Compare November 7, 2024 13:28

compress binary

a98566c

cedric-cordenier force-pushed the CAPPL-3-ph branch from 6eba838 to a98566c Compare November 7, 2024 14:00

cedric-cordenier force-pushed the CAPPL-3-ph branch from 2744b82 to cd8c663 Compare November 7, 2024 15:35

GetUnfinished fix

a69ca05

cedric-cordenier added 6 commits November 15, 2024 16:41

Bump common

2b3af8b

Update workflow name

510bfc0

Add second workflow

54faaf4

Update registry to run two workflows; add updated workflows

58b51f9

Update artifacts

30bb2f9

Merge branch 'develop' into CAPPL-3-ph

0bcb61f

cedric-cordenier force-pushed the CAPPL-3-ph branch from 95c1210 to 0bcb61f Compare November 20, 2024 18:59

cedric-cordenier added 4 commits November 20, 2024 19:00

Update go.sum

13034bd

Update workflows

43ac99c

Update PoR workflow with timestamp adjustment

c5f0bf1

Update workflows

d65ddec

cedric-cordenier force-pushed the CAPPL-3-ph branch from 0b2fe1d to d65ddec Compare November 21, 2024 22:08

justinkaseman and others added 8 commits November 21, 2024 15:07

merge develop

2c79077

Revert "chore(workflows): stub out event handlers (#15313)"

93b05ee

This reverts commit 86ab654.

Revert "feat(workflows): adds registry syncer (#15277)"

1608708

This reverts commit 64aeef9.

Update hardcoded wasm workflows

630193a

Update workflows

f5e6fe5

One more update

22c235a

One more update

cc98bbd

Update hardcoded workflow with CLI generated binary

9214ffc

justinkaseman added build-publish Build and Publish image to SDLC and removed build-publish Build and Publish image to SDLC labels Nov 27, 2024

justinkaseman added 7 commits November 27, 2024 09:30

Revert "feat(workflows/handler): adds all event handlers (#15400)"

7856854

This reverts commit ae63cc6.

Revert "feat(handler): implements handle workflow registered event (#…

2a691fe

…15383)" This reverts commit fa78941.

Revert "feat(workflows): adds orm methods for managing specs (#15356)"

4af21d5

This reverts commit 1168674.

Revert "chore(workflows): stub out event handlers (#15313)"

aafd7bd

This reverts commit 86ab654.

Revert "feat(workflows): adds registry syncer (#15277)"

68401d8

This reverts commit 64aeef9.

Merge branch 'develop' into CAPPL-3-ph

f8a7688

Revert "[CAPPL-316] implement FetchFunc (#15424)"

4da22ce

This reverts commit 935612e.

cedric-cordenier closed this Dec 2, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Cappl 3 ph #15125

Cappl 3 ph #15125

patrickhuie19 commented Nov 5, 2024

github-actions bot commented Nov 5, 2024 •

edited

Loading

github-actions bot commented Nov 6, 2024 •

edited

Loading

github-actions bot commented Nov 7, 2024

Cappl 3 ph #15125

Cappl 3 ph #15125

Conversation

patrickhuie19 commented Nov 5, 2024

Requires

Supports

github-actions bot commented Nov 5, 2024 • edited Loading

Static analysis results are available

github-actions bot commented Nov 6, 2024 • edited Loading

AER Report: CI Core

1. Redeclared variables in test file: [Run tests with flakeguard]

2. fmt.Printf format error: [Run tests]

3. Replacement transaction underpriced: [Run tests]

4. CSA key does not exist: [Run tests with flakeguard]

5. SQL transaction aborted: [Run tests with flakeguard]

6. Post request EOF error: [Run tests with flakeguard]

AER Report: Operator UI CI ran successfully ✅

github-actions bot commented Nov 7, 2024

Flaky Test Detector for ./go.mod project has failed ❌

Failed Tests

github-actions bot commented Nov 5, 2024 •

edited

Loading

github-actions bot commented Nov 6, 2024 •

edited

Loading

Flaky Test Detector for `./go.mod` project has failed ❌